forked from calcom/cal.com
-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[pull] main from calcom:main #2
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* fix: trigger createOrUpdateMemberships from dsync so SAML signup triggers side-effects * forEach does not wait for async/await done-ness
* fix: event-type atom tabs controlled by prop * fixup! fix: event-type atom tabs controlled by prop
* fix syntax for code examples * fixup
* fix: booker timezone from preference instead of session * fixup! fix: booker timezone from preference instead of session
* only fetch bookings of this month for weighted rr * test set up * only get bookings of current month * reverts test setup * first changes for weight adjustments * reset booking count + adjust calibration * depreciate weightAdjustment * get only bookings created this month * fix typo * make sure createdAt for hosts is correct * use earliest possibel date as fallback * add missing createdAt date to tests * fix typo * clean up changes in tests * fix typo * change end date to current date * fix: Fall back to empty host array when no hosts are found * fix: Restructure code a little * fixed test, incorrectly used now outdated var * perf: remove Dayjs from getLuckyUser * Refactor getHostsWithCalibration for optimised performance, intentionally break test as findMany is always an array * Better mock for host.findMany * Remove team-event-types.test.ts, move to appropriate package * TypeScript cannot auto-infer that an array is non-empty when assigning to a var * fix: Type Fixes and DistributionMethod enum add * Optimise tests * Added test to show that bookings made before a newHost was added affect the lucky user result * Throw error when the usersWithHighestPriority is empty, which should never happen --------- Co-authored-by: CarinaWolli <[email protected]> Co-authored-by: Alex van Andel <[email protected]>
…member. (#17513) * fix due to error in zod.parse while adding new member * updated zod parsing * fix the linking with parent * update fixture to accomodate assignAllTeamMembers for team and org scenarios * added e2e test * nit
* chore: add sentry performance tracking api-v2 * fixup! chore: add sentry performance tracking api-v2
* move search to severside * transform zod stirng to lowercase
* adding input types and method to controller/service * add output types * add outputs and inputs * feat: reassign automatically * feat: reassign to user * e2e test for reassign * update platform libraries + experimental modules for jest * fix test --------- Co-authored-by: supalarry <[email protected]> Co-authored-by: Lauris Skraucis <[email protected]>
…ams (#17530) * adds attendee and host names in forwarded params * address feedback * add guest emails
* fixup * resolve merge conflicts * fixup fixup * trying some fixups
* fix syntax for code examples * update iframe styling
* feat: Delete attribute modal * move Modal file --------- Co-authored-by: sean-brydon <[email protected]>
Co-authored-by: Udit Takkar <[email protected]>
* fix: remove from host when removed from org * test: add test for removing members * fix: type errors, some unused params * fix: test * refactor: use prisma.count --------- Co-authored-by: Alex van Andel <[email protected]>
…uteId (#17867) ## What does this PR do? <!-- Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change. --> We were parsing routes based on `eventTypeId` but routes could be pointing to the same event type. This was causing misroutings. - Fixes #XXXX (GitHub issue number) - Fixes CAL-XXXX (Linear issue number - should be visible at the bottom of the GitHub issue description) <!-- Please provide a loom video for visual changes to speed up reviews Loom Video: https://www.loom.com/ --> ## Mandatory Tasks (DO NOT REMOVE) - [x] I have self-reviewed the code (A decent size PR without self-review might be rejected). - [ ] I have updated the developer docs in /docs if this PR makes changes that would require a [documentation change](https://cal.com/docs). If N/A, write N/A here and check the checkbox. - [ ] I confirm automated tests are in place that prove my fix is effective or that my feature works.
Co-authored-by: Amit Sharma <[email protected]> Co-authored-by: Hariom <[email protected]> Co-authored-by: Syed Ali Shahbaz <[email protected]>
remove comment
…e in embed-react package (#17790) * Update vite.config.js * linting fix --------- Co-authored-by: Hariom <[email protected]>
…integ… (#17903) * Added proper updates/interactivity on the "Getting started" app integrations. - InstallAppButtonWithoutPlanCheck properly vends the mutation data in the render's props - Fixed the improper props wiring in the AppConnectionItem - Added a loading state for reactivity * -Added toaster for error/success confirmation
* feat: return noShowGuest data from eventsByStatus & eventsTimeline procedures * feat: add noShowGuest Data in BookingKPICards * feat: add noShowGuest Data in BookingStatusLineChart * feat: track noShowGuests in outer query * chore: fix noShowGuest naming
* fix: fix UI regression regarding DataTable * update comment
… any static app (#17840) * fix: meetingurl does not save when we change location * update * fix type error --------- Co-authored-by: Alex van Andel <[email protected]>
* fix: manually encode special characters * fix test * fix * fix
* remove heading for more space * fix: mobile support for mass attribute assigning bar * change the order of Members item on Navigation * refactor NavigationItem to support moreOnMobile property * fix logic for nav * decrease column widths on member list * improve mobile design for mass attribute assign * adjust mobile ui for DataTableSelectionBar * add key to EventTypesList --------- Co-authored-by: Peer Richelsen <[email protected]> Co-authored-by: Anik Dhabal Babu <[email protected]>
Co-authored-by: Alex van Andel <[email protected]>
* fix: display option label int the routing form response email * Update * fix type error * type error
…nt (#17761) * Show 'Add to calendar' section for customers after booking an event * removed comment --------- Co-authored-by: Prashant Varma <[email protected]>
* remove ai translations description for atoms * PR feedback * fixup --------- Co-authored-by: Benny Joo <[email protected]> Co-authored-by: Somay Chauhan <[email protected]> Co-authored-by: Morgan <[email protected]>
* fix: fix date comparison for OOO * fix date comparison with today, and add tests * remove wrong code * remove logic that is no longer in use * fix type error * mock more methods --------- Co-authored-by: Alex van Andel <[email protected]>
* fix: Date Range Picker behaviour * remove console log * update * update * update css --------- Co-authored-by: Peer Richelsen <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )